Skip to content

prevent failure in flaky spec reporting when running from external PR#23845

Merged
toy merged 1 commit into
opf:devfrom
toy:flaky-test-comment-for-everyone
Jun 24, 2026
Merged

prevent failure in flaky spec reporting when running from external PR#23845
toy merged 1 commit into
opf:devfrom
toy:flaky-test-comment-for-everyone

Conversation

@toy

@toy toy commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

I didn't find a simple way to allow gh pr comment for external PR and knowing about flaky specs is important for maintainers, not contributors.

This fixes following error (https://github.com/opf/openproject/actions/runs/27945332159/job/82688649716?pr=23811):
image

Error in downstream-ci / SaaS tests (pull_request) is not relevant, fixing not seeing error message in #23882, will fix the error itself separately

Comment thread .github/workflows/test-core.yml Fixed
@toy toy force-pushed the flaky-test-comment-for-everyone branch from c7a99f1 to 4409660 Compare June 22, 2026 13:47
@toy toy requested a review from thykel June 22, 2026 13:49
@toy toy force-pushed the flaky-test-comment-for-everyone branch from 9d4857f to 4409660 Compare June 23, 2026 13:12

@thykel thykel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀 🙏

@toy toy merged commit fbfbfa9 into opf:dev Jun 24, 2026
24 of 27 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 24, 2026
@toy toy deleted the flaky-test-comment-for-everyone branch June 24, 2026 10:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants